Skip to content

fix(material/button): prevents mat-icon being cut off by text-spacing #30891

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 4, 2025

Conversation

essjay05
Copy link
Contributor

Updates Angular Components Button component specifically for .mat-icon buttons to add a min-height of min-content to avoid the icon from being cut off on the bottom when text-spacing is applied.

Fixes b/411228600

BEFORE screenshot
AFTER screenshot

…ng vertically

Updates Angular Components Button component specifically for .mat-icon buttons to
add a min-height of min-content to avoid the icon from being cut off on the bottom
when text-spacing is applied.

Fixes b/250063405
Updates previous changes to Angular Components buttons containing
icons and adds flex-shrink: 0; to prevent the sides from being
cut off when text-spacing is applied or on smaller screens.

Fixes b/411228600
@essjay05 essjay05 requested a review from a team as a code owner April 17, 2025 00:11
@essjay05 essjay05 requested review from adolgachev and andrewseguin and removed request for a team April 17, 2025 00:11
Adds comment to fix lint error.
@adolgachev adolgachev added Accessibility This issue is related to accessibility (a11y) dev-app preview When applied, previews of the dev-app are deployed to Firebase action: global presubmit The PR is in need of a google3 global presubmit labels May 16, 2025
Copy link

Deployed dev-app for d22b5f1 to: https://ng-dev-previews-comp--pr-angular-components-30891-dev-bqnj40tv.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@adolgachev adolgachev removed the request for review from andrewseguin June 2, 2025 19:34
@adolgachev adolgachev added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release and removed action: global presubmit The PR is in need of a google3 global presubmit labels Jun 2, 2025
@adolgachev adolgachev merged commit 8523397 into angular:main Jun 4, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility This issue is related to accessibility (a11y) action: merge The PR is ready for merge by the caretaker area: material/button dev-app preview When applied, previews of the dev-app are deployed to Firebase target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants